Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix PHPLDAPADMIN_LDAP_HOSTS to treat as a valid environment variable. #24

Closed
wants to merge 1 commit into from
Closed

Fix PHPLDAPADMIN_LDAP_HOSTS to treat as a valid environment variable. #24

wants to merge 1 commit into from

Conversation

shinnya
Copy link

@shinnya shinnya commented Aug 21, 2016

Hi. I found that I set PHPLDAPADMIN_LDAP_HOSTS environment variable, but it didn't reflect my setting. I think image/service/phpldapadmin/startup.sh doesn't handle the environment variable properly because $ is missing.

Best Regards,

@osixia
Copy link
Collaborator

osixia commented Aug 21, 2016

Hi,
Sorry this may not fix you issue. Please refer to:
https://github.com/osixia/docker-light-baseimage/blob/stable/image/tool/complex-bash-env

Can you describe your probleme instead ?

@shinnya
Copy link
Author

shinnya commented Aug 21, 2016

Thank you for your quick reply! Uhmm, It seems that I misunderstood complex-bash-env.

My problem was that I created my container with docker-compose and couldn't configure /var/www/phpldapadmin/config/config.php properly. When I executed docker-compose up -d multiple times without shutting down my containers by docker-compose down, the following settings listed in README weren't generated.

$servers->newServer('ldap_pla');
$servers->setValue('server','name','ldap.example.org');
$servers->setValue('server','host','ldap.example.org');
$servers->setValue('server','tls',true);
$servers->setValue('login','bind_id','cn=admin,dc=example,dc=org');
$servers->newServer('ldap_pla');
$servers->setValue('server','name','ldap2.example.org');
$servers->setValue('server','host','ldap2.example.org');
$servers->newServer('ldap_pla');
$servers->setValue('server','name','ldap3.example.org');
$servers->setValue('server','host','ldap3.example.org');

Alternatively I got the following:

{{ PHPLDAPADMIN_SERVERS }}

Anyway, now I've succeded in setting up my LDAP server correctly. Thank you.

@shinnya shinnya closed this Aug 21, 2016
@shinnya shinnya deleted the fix/PHPLDAPADMIN_LDAP_HOSTS_missing branch August 21, 2016 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants